Skip to content

feat: implement model-specific suffix resolution for M3 mtp as "EAGLE"#469

Merged
adibarra merged 1 commit into
masterfrom
feat/model-specific-suffix-resolution
Jun 18, 2026
Merged

feat: implement model-specific suffix resolution for M3 mtp as "EAGLE"#469
adibarra merged 1 commit into
masterfrom
feat/model-specific-suffix-resolution

Conversation

@adibarra

@adibarra adibarra commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Note

Low Risk
Display-label only; omitting model on getHardwareConfig keeps prior MTP behavior, with tests covering cache isolation.

Overview
MiniMax-M3 runs stored with spec_method / hwKey token mtp now show EAGLE (e.g. suffix (vLLM, M3 EAGLE)) instead of generic MTP. Other models are unchanged.

Adds resolveFrameworkPartLabel and MODEL_SPEC_METHOD_LABELS in @semianalysisai/inferencex-constants, and extends getHardwareConfig(hwKey, model?) so suffix parts respect per-model overrides. Model-scoped cache keys avoid leaking M3 labels into model-less lookups.

Call sites pass the display model (or row model) into getHardwareConfig across inference charts, tables, CSV export, eval labels, submissions, changelog config keys, calculator throughput, and AI chart bars. SubmissionsTable and eval/changelog builders use the same resolver for spec-method cells and headers.

Reviewed by Cursor Bugbot for commit 1c96c70. Bugbot is set up for automated code reviews on this repo. Configure here.

@adibarra adibarra marked this pull request as ready for review June 18, 2026 01:41
@vercel

vercel Bot commented Jun 18, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
inferencemax-app Ready Ready Preview, Comment Jun 18, 2026 1:41am

Request Review

@adibarra adibarra merged commit 6ee448c into master Jun 18, 2026
24 checks passed
@adibarra adibarra deleted the feat/model-specific-suffix-resolution branch June 18, 2026 01:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant